Fix inconsistencies on example topologies and add backward compatibility to Port.vlan_range #141
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related to atlanticwave-sdx/pce#200
Description of the change
This pull request fix a few inconsistencies on the example topologies (
amlight.json
,sax.json
andsdx.json
). Most of the inconsistencies consist of:amlight
->amlight.net
-- correct in some places, wrong in others)node": "urn:sdx:port:sax:B1",
->"node": "urn:sdx:node:sax:B1"
)"node": "A1:2"
->"node": "urn:sdx:node:zaoxi:A1",
)Because of those errors, the breakdowns failed to be computed, as well as VLAN reservations and other cases.
Additionally, I've also added backward compatibility for Port.vlan_range on Topology v1: if the
vlan_range
is not defined on the port butlabel_range
is, PortHandler will leverage the former.